Skip to content

fix(profile): end a barless profile's scroll at Leave Chat - #1037

Merged
bmc08gt merged 3 commits into
mainfrom
fix/profile-barless-scroll-clearance
Oct 10, 2026
Merged

bmc08gt merged 3 commits into
mainfrom
fix/profile-barless-scroll-clearance

Conversation

@bmc08gt

@bmc08gt bmc08gt commented Oct 10, 2026

Copy link
Copy Markdown
Collaborator

Opened from the chat, a group profile long enough to scroll left about 80pt of empty space under Leave Chat. The pinned bar's 56pt clearance stayed on with no bar to clear, under the content's own 24pt padding.

  • A profile without a pinned bar takes no clearance. Leave Chat ending the scroll sits where the pinned bar puts it, overlapping the home indicator's inset by 12pt.
  • The barless fade covers only the home indicator's inset, so it no longer draws over Leave Chat. The DM profile's encryption line gets the same clearance and fade.
  • ProfileScrollFit drops its offset for a clearance that turned on with overflow, since none does now.
  • Both profile screens hide the scroll indicator.

Opened from the chat, a group profile long enough to scroll left about
80pt of empty space under Leave Chat: the pinned bar's 56pt clearance
stayed on with no bar to clear, under the content's own 24pt padding.

A profile without a pinned bar now takes no clearance. Leave Chat ending
the scroll sits where the pinned bar puts it, overlapping the home
indicator's inset by 12pt, and the barless fade covers only that inset so
it no longer dims the button. The DM profile's encryption line gets the
same clearance and fade.

ProfileScrollFit no longer offsets for a clearance that turned on with
overflow, since none does.
@bmc08gt bmc08gt self-assigned this Oct 10, 2026
@bmc08gt
bmc08gt merged commit c6324d8 into main Oct 10, 2026
2 checks passed
@bmc08gt
bmc08gt deleted the fix/profile-barless-scroll-clearance branch October 10, 2026 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant